Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(image): add image preview description template #16500

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

denyssalon
Copy link

@denyssalon denyssalon commented Oct 7, 2024

Feature Requests

This PR extends image component with custom image preview description template
Screenshot 2024-10-07 at 13 34 34

Copy link

vercel bot commented Oct 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Oct 7, 2024 11:32am
primeng-v18 ⬜️ Ignored (Inspect) Visit Preview Oct 7, 2024 11:32am

@denyssalon denyssalon changed the title feat(image): add image description template feat(image): add image preview description template Oct 7, 2024
@@ -29,4 +29,9 @@ export interface ImageTemplates {
* Custom template of closeicon.
*/
closeicon(): TemplateRef<any>;

/**
* Custom template of imagedescription.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

space here? image description

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SoyDiego It is a name of the template which is imagedescription (without space).
I made it the same way as comments above (closeicon, rotaterighticon etc). Do you still want me to put a space there?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh ok, sorry, thanks for re check!
Now we should wait if PrimeNG Team approve it!

@denyssalon
Copy link
Author

Hello @SoyDiego. Do you think this has a chance to be merged someday?

@SoyDiego
Copy link
Contributor

Hello @SoyDiego. Do you think this has a chance to be merged someday?

Sorry, i'm not part of PrimeNG Team, only a contributor like you :).
They will do it for sure, but sometimes we need to wait because they are working in the v18

@mertsincan
Copy link
Member

mertsincan commented Nov 18, 2024

Hi @denyssalon,

Thanks a lot for your contribution! Can you create an issue and link it? Then, I'll merge it asap. Also, could you please create a PR for v18 as well?

@mertsincan mertsincan added the Resolution: Needs Issue An issue needs to be created for the pull request label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Needs Issue An issue needs to be created for the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants